fix(claude-code): skip primary + duplicate banks in recallAdditionalBanks - #2625

Fixes #2604.

Problem

The claude-code recall hook's recallAdditionalBanks loop recalled every entry with no dedup against the session's resolved primary bank. So a bidirectional cross-bank setup (each bank lists the other, so both must appear in recallAdditionalBanks) re-recalled the primary on every prompt — one wasted recall call (~0.4–0.65s) per user prompt, plus duplicate context.

Fix

Guard the additional-banks loop with a seen_banks set seeded with the primary bank_id (already recalled above). This skips the primary and also de-dups repeated entries in the list.

seen_banks = {bank_id}
for extra_bank_id in additional_banks:
    if extra_bank_id in seen_banks:
        continue
    seen_banks.add(extra_bank_id)
    ...

Test

Adds test_additional_banks_skip_primary_and_duplicates — primary listed in recallAdditionalBanks plus a duplicated extra → asserts the primary is recalled once, the extra once, 2 calls total (not 4). Full test_hooks.py green (30 passed).

Thanks @hb-cam for the precise repro and the one-liner.